Skip to content
New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

Removed grouping functionality from front-end #37

Merged
merged 3 commits into from
Nov 18, 2015
Merged

Removed grouping functionality from front-end #37

merged 3 commits into from
Nov 18, 2015

Conversation

xaun
Copy link
Contributor

@xaun xaun commented Nov 12, 2015

Switching to and from comparison mode now triggers normal metadata update cycle, with api returning formatted accounts data. The params-checkboxes settings directive also now only converts relevant data to metadata.

…e cycle

Switching to and from comparison mode now triggers normal metadata update cycle, with api returning formated accounts data. The params-checkboxes settings directive also now only converts relevant data to metadata.
@xaun
Copy link
Contributor Author

xaun commented Nov 12, 2015

https://github.com/maestrano/impac/pull/102
@cesar-tonnoir just leaving these PR's here for you to take a look at this morning. It is working seamlessly from my testing on Mno across single and multi org, but this is without the behaviour changes mentioned here: #6.

@cesar-tonnoir
Copy link
Contributor

When I unselect all the organizations to simulate a "data not found", I have this error:

TypeError: Cannot read property 'length' of null
    at Object.w.initContext (impac-angular.js?body=1:3402)
    at impac-angular.js?body=1:2470
    at angular.js:14634
    at m.a.$get.m.$eval (angular.js:15878)
    at m.a.$get.m.$digest (angular.js:15689)
    at m.a.$get.m.$apply (angular.js:15986)
    at g (angular.js:10511)
    at L (angular.js:10683)
    at XMLHttpRequest.A.onload (angular.js:10624)

and the widget keeps loading

@cesar-tonnoir
Copy link
Contributor

when I select "compare matching accounts across your companies", it lets me select any account, and not only the accounts existing in the two companies

screenshot

@xaun
Copy link
Contributor Author

xaun commented Nov 12, 2015

@cesar-tonnoir thanks for testing - those two bugs should be fixed, if you could give them another test on your environment and let me know. Cheers

@@ -22,43 +22,29 @@ module.controller('WidgetAccountsComparisonCtrl', ($scope, $q, ChartFormatterSvc
# --------------------------------------
w.initContext = ->
$scope.isDataFound = w.content? && !_.isEmpty(w.content.complete_list)
# parameters for params-checkboxes.directive
# defines the available options for params-checkboxes.directive
$scope.comparisonModeOptions = [{
id: 'compare_accounts',
label: 'Compare matching accounts across your companies',
value: false,
onChangeCallback: $scope.multiCompanyComparisonOnChange
Copy link
Contributor

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

@xaun: can be changed to "$scope.updateSettings()" instead of defining a new function

cesar-tonnoir added a commit that referenced this pull request Nov 18, 2015
Removed grouping functionality from front-end
@cesar-tonnoir cesar-tonnoir merged commit 2b0bf8d into maestrano:1.0 Nov 18, 2015
@xaun xaun deleted the rework/accounts-comparison branch November 18, 2015 05:47
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
None yet
Projects
None yet
Development

Successfully merging this pull request may close these issues.

2 participants